Skip to content

Fix Dead links to documents in the comments#32619

Closed
Naturalclar wants to merge 3 commits into
facebook:mainfrom
Naturalclar:fix/deadLinksToDocs
Closed

Fix Dead links to documents in the comments#32619
Naturalclar wants to merge 3 commits into
facebook:mainfrom
Naturalclar:fix/deadLinksToDocs

Conversation

@Naturalclar
Copy link
Copy Markdown
Contributor

Summary

Links under reactnative.dev that ended with .html lead to Page not found.
Fixed the url so that users get sent to the appropriate url.

Changelog

[General] [Fixed] - Fixed dead links in the comments.

Test Plan

  • Changed links are accessible

@Naturalclar Naturalclar requested a review from hramos as a code owner November 18, 2021 08:21
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. labels Nov 18, 2021
@analysis-bot
Copy link
Copy Markdown

analysis-bot commented Nov 18, 2021

Platform Engine Arch Size (bytes) Diff
ios - universal n/a --

Base commit: 26e30a5
Branch: main

@analysis-bot
Copy link
Copy Markdown

analysis-bot commented Nov 18, 2021

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 8,477,119 -703
android hermes armeabi-v7a 7,776,517 -517
android hermes x86 8,945,994 -602
android hermes x86_64 8,890,143 -525
android jsc arm64-v8a 9,791,933 -688
android jsc armeabi-v7a 8,752,535 -514
android jsc x86 9,740,749 -586
android jsc x86_64 10,341,651 -510

Base commit: 26e30a5
Branch: main

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@cortinico has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Comment thread Libraries/Vibration/Vibration.js Outdated
* See https://reactnative.dev/docs/vibration#vibrate
*/
vibrate: function (
vibrate: function(
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change should be amended as it's making the CI fail

Comment thread Libraries/Vibration/Vibration.js Outdated
* See https://reactnative.dev/docs/vibration#cancel
*/
cancel: function () {
cancel: function() {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same as above

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cortinico thanks for the review! fixed the format

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@cortinico has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Nov 22, 2021
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@cortinico merged this pull request in 00bb2ba.

@Naturalclar Naturalclar deleted the fix/deadLinksToDocs branch November 26, 2021 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants